Skip to content

feat(scripts): add diff:flat script #25352

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 70 commits into
base: main
Choose a base branch
from
Draft

feat(scripts): add diff:flat script #25352

wants to merge 70 commits into from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Dec 9, 2024 •
edited
Loading

Summary

Adds a new script to show changes in a way that doesn't omit any changes, while keeping the diff small.

(This works especially well if multiple features are set [from the same version] to the same version.)

Test results and supporting details

I have been sharing screenshots in PRs.

Usage:

npm run diff:flat [<pr-number>]
npm run diff:flat <head-ref> [<base-ref>]

Related issues

caugner added 30 commits December 2, 2024 12:23
@caugner
chore(scripts): add flat-diff
a921242
@caugner
chore(flat-diff): flatten arrays
b503acc
@caugner
chore(flat-diff): convert notes to array
0f315ca
@caugner
chore(flat-diff): improve HTML rendering
3e7c008
@caugner
fix(diff-flat): show booleans
d86cd73
@caugner
enhance(flat-diff): hide new mirror
871bcbc
@caugner
enhance(flat-diff): arrayify support statements
777eacf
@caugner
enhance(flat-diff): stringify flags
37cc7fb
@caugner
style: fix lints
f0fbc35
@caugner
enhance(diff-flat): add --group option
cf432c7 
Groups by change.
@caugner
fix(flat-diff): group values by browser and field
86bed0b
@caugner
enhance(diff-flat): align suffix
8e6e181
@caugner
enhance(diff-flat): group groups
3642e18 
Group keys by values, rather than keys by value.
@caugner
fix(diff-flat): print grouped values before keys
c10c106
@caugner
enhance(diff-flat): annotate flags
700fe48
@caugner
enhance(diff-flat): merge versions + prefix/flags/altname
08db344
@caugner
enhance(diff-flat): improve group formatting
118405a
@caugner
enhance(diff-flat): collapse groups + order by value
0eb3d60
@caugner
enhance(diff-flat): omit "__compat"/"support" + trailing placeholder
5611d24
@caugner
enhance(flat-diff): join tags
7f17aa7
@caugner
enhance(diff-flat): diff value again
7fdbe60
@caugner
fix(diff-flat): split at footnotes
3712dfb
@caugner
refactor(diff-flat): inline print()
a069adc
@caugner
enhance(diff-flat): print refs in context
06bfed8
@caugner
chore(deps-dev): add strip-ansi 7.1.0
c845b50 
# Conflicts:
#	package-lock.json
@caugner
fix(diff-flat): sort ignoring ANSI escape codes
879d074
@caugner
enhance(diff-flat): serialize status
645d3b6
@caugner
fix(diff-flat): improve word splitting
e1173cd
@caugner
fix(diff-flat): remove inverse key sort
6bb510d
@caugner
fix(diff-flat): call diffKeys() with correct order
8f45778
@github-actions GitHub Actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added size:l [PR only] 101-1000 LoC changed and removed merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. size:xl [PR only] >1000 LoC changed labels Mar 28, 2025
@caugner caugner mentioned this pull request Apr 9, 2025
Merged
@github-actions github-actions bot added the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label Apr 9, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 9, 2025

This pull request has merge conflicts that must be resolved before it can be merged.

@caugner caugner force-pushed the flat-diff branch 2 times, most recently from d0f36c6 to 18ff480 Compare May 15, 2025 17:09
@caugner
fix(scripts): avoid execSync() in favor of spawn()
e9d478d 
Deprecates the `exec()` and `execAsync()` utility functions.
@github-actions github-actions bot removed the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label May 23, 2025
@caugner caugner mentioned this pull request May 23, 2025
Merged
@github-actions github-actions bot added the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label May 30, 2025
@github-actions GitHub Actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency package or file. infra Infrastructure issues (npm, GitHub Actions, releases) of this project scripts Issues or pull requests regarding the scripts in scripts/. size:l [PR only] 101-1000 LoC changed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@caugner